Skip to content

Unify button look and rearrange buttons to make them more compact #712

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 14, 2025

Conversation

corebonts
Copy link
Contributor

  • Unify icons (fontawesome public domain fonts)
  • Unify button sizes and alignment
  • Make the button arrangement beside the input more compact
  • Add icon for the menu-trigger and fix it's alignment
  • Fix the wrapping of the title (it overflown when using ollama sha-named model files)

image

@cjpais
Copy link
Collaborator

cjpais commented Mar 13, 2025

Thanks! This will likely make it into the next release when I do my testing for it and see how it compares to the current. As far as I can see from the code and screenshot it will make it.

Will need to check on the icons, could you give the link to the source?

@corebonts
Copy link
Contributor Author

Sure! They are part of this collection: https://fontawesome.com/search?o=r&ic=free&s=solid&ip=classic
License: https://fontawesome.com/license/free

Attribution is required by MIT, SIL OFL, and CC BY licenses. Downloaded Font Awesome Free files already contain embedded comments with sufficient attribution, so you shouldn't need to do anything additional when using these files normally.

@corebonts
Copy link
Contributor Author

corebonts commented Mar 13, 2025

I just learned that the "completion" UI has different buttons so I also modify that.

...done
image

- images
- button styles
- textarea style
@cjpais
Copy link
Collaborator

cjpais commented Mar 14, 2025

looks great, thanks

@cjpais cjpais merged commit c344ac7 into Mozilla-Ocho:main Mar 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants